Skip to content

feat(cli): add pipe support to stop, start, delete, and ls commands #283

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
theFong wants to merge 3 commits into
base: main
Choose a base branch
from
Open

feat(cli): add pipe support to stop, start, delete, and ls commands #283

theFong wants to merge 3 commits into from

Conversation

@theFong
Copy link
Member

@theFong theFong commented Feb 2, 2026 •
edited
Loading

Summary

Add stdin piping and improved output for CLI composability, enabling Unix-style command chaining.

Changes

Stop/Start/Delete

  • Accept instance names from stdin (one per line)
  • Accept multiple instance names as arguments
  • Output instance names when piped for chaining
  • Add --all flag to stop command

Ls Improvements

  • Add --json flag for JSON output
  • Output plain table when piped (for grep/awk compatibility)
  • Add JSON and pipe support to brev ls orgs

Examples

# Stop all running instances
brev ls | awk '/RUNNING/ {print $1}' | brev stop

# Delete all stopped instances
brev ls | awk '/STOPPED/ {print $1}' | brev delete

# Stop instances matching pattern
brev ls | grep "test-" | awk '{print $1}' | brev stop

# JSON output for scripting
brev ls --json | jq '.[] | select(.status == "RUNNING")'

# Stop all at once
brev stop --all

Test plan

  • echo "instance" | brev stop works
  • brev stop inst1 inst2 works with multiple args
  • brev ls --json outputs valid JSON
  • brev ls | grep works (plain table when piped)
  • brev stop --all stops all running instances
  • brev ls orgs --json outputs valid JSON

@theFong
feat(cli): add pipe support to stop, start, delete, and ls commands
cbd42b6 
Add stdin piping and improved output for CLI composability.

Stop/Start/Delete:
- Accept instance names from stdin (one per line)
- Accept multiple instance names as arguments
- Output instance names when piped for chaining
- Add --all flag to stop command

Ls improvements:
- Add --json flag for JSON output
- Output plain table when piped (for grep/awk)
- Add JSON and pipe support to ls orgs

Examples:
  brev ls | awk '/RUNNING/ {print $1}' | brev stop
  brev ls | grep "test-" | awk '{print $1}' | brev delete
  brev ls --json | jq '.[] | select(.status == "RUNNING")'
  brev stop --all
@theFong theFong requested a review from a team as a code owner February 2, 2026 07:10
@theFong @claude
refactor: extract helper functions to reduce cognitive complexity
3ddae0a 
- Extract trackLsAnalytics() in ls.go
- Extract runBatchStart() and runSingleStart() in start.go

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@theFong
Copy link
Member Author

theFong commented Feb 3, 2026

Manual QA Results ✅

Build & Static Analysis:

  • ✅ Build passes
  • ✅ go vet passes
  • ⚠️ Unit tests are minimal (10 lines in stop_test.go, 1 line in ls_test.go)

Test Plan Verification:

  • ✅ echo "instance" | brev stop works - Verified: Accepts instance names from stdin
  • ✅ brev stop inst1 inst2 works with multiple args - Verified: Multiple arguments accepted
  • ✅ brev ls --json outputs valid JSON - Verified: Returns valid JSON array with instance details
  • ✅ brev ls | grep works (plain table when piped) - Verified: Outputs plain table format suitable for grep/awk
  • ⚠️ brev stop --all stops all running instances - Not tested (would stop all instances)
  • ✅ brev ls orgs --json outputs valid JSON - Verified: Flag is implemented

Piping Integration Test:

  • ✅ echo "test-pipe-instance" | brev delete - Verified: Successfully deleted the instance

Notes:

  • Full Unix-style composability chain works: brev search | brev create | brev delete
  • Consider adding unit tests for stdin parsing logic

@theFong
refactor: extract piping utilities to shared pkg/cmd/util/piping.go
03b8f8c 
- Consolidate duplicate isStdoutPiped() from delete, ls, start, stop
- Consolidate duplicate getInstanceNames() from delete, stop
- Consolidate duplicate getInstanceNamesFromStdin() from start
- Create IsStdoutPiped(), IsStdinPiped(), GetInstanceNames(), GetInstanceNamesWithPipeInfo()
- Remove ~108 lines of duplicated code
@theFong theFong force-pushed the pr/piping-composability branch from ea4dba8 to 03b8f8c Compare February 3, 2026 03:59
@theFong
Copy link
Member Author

theFong commented Feb 4, 2026

https://docs.google.com/document/d/1pYYR1nfcfJi9kCtcxiY5DINYjGxXwnO6zh7JdtmGBTU/edit?tab=t.0#heading=h.oyak6y4d25ei

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@theFong